Skip to content

Conversation

ydah
Copy link
Member

@ydah ydah commented Feb 12, 2025


Before submitting the PR make sure the following are checked:

  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • [-] Added tests.
  • [-] Updated documentation.
  • [-] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
  • The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

ydah added 2 commits February 12, 2025 16:42
```
❯ bundle exec rubocop
Inspecting 35 files
..............C....................

Offenses:

lib/rubocop/rspec_rails/description_extractor.rb:60:11: C: [Correctable] Style/MultipleComparison: Avoid comparing a variable with multiple items in a conditional, use Array#include? instead.
          yardoc.superclass.path == RSPEC_RAILS_COP_CLASS_NAME || ...
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/rubocop/rspec_rails/description_extractor.rb:60:39: C: [Correctable] Layout/TrailingWhitespace: Trailing whitespace detected.
          yardoc.superclass.path == RSPEC_RAILS_COP_CLASS_NAME ||
                                      ^
lib/rubocop/rspec_rails/description_extractor.rb:61:15: C: [Correctable] Layout/LineLength: Line is too long. [95/80]
            yardoc.superclass.path == RUBOCOP_COP_CLASS_NAME
              ^^^^^^^^^^^^^^^
lib/rubocop/rspec_rails/description_extractor.rb:60:41: C: [Correctable] Layout/ArrayAlignment: Align the elements of an array literal if they span more than one line.
          yardoc.superclass.path == RSPEC_RAILS_COP_CLASS_NAME ||
                                        ^^^^^^^^^^^^^^^^^^^^^^

35 files inspected, 4 offenses detected, 4 offenses autocorrectable
```
@ydah ydah requested a review from a team as a code owner February 12, 2025 07:44
@bquorning bquorning merged commit 2ffe982 into master Feb 12, 2025
27 checks passed
@bquorning bquorning deleted the fixes branch February 12, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants